Skip to content

docs: merge plugin-related navigation items #5655

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

gtsiolis
Copy link
Contributor

@gtsiolis gtsiolis commented Jul 7, 2020

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Underlying tools
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

Other information:

Fix #5652.

items:
[
{ text: 'Info', link: '/dev-guide/ui-info.md' },
{ text: 'API', link: '/dev-guide/ui-api.md' },
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully this duplicate label here won't create any confusion given there's a section label above in https://github.com/vuejs/vue-cli/pull/5655/files#diff-d669c10d59b56bb12a815dd48ce8d884R100.

Maybe it's worth considering refactoring these closely related documentation sections. 🔮

@haoqunjiang haoqunjiang merged commit 8f5691b into vuejs:dev Jul 20, 2020
@gtsiolis
Copy link
Contributor Author

Thanks @sodatea! 🏀

@gtsiolis gtsiolis deleted the 5652-merge-plugin-related-navigation-items branch August 4, 2020 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Merge plugin-related navigation items
2 participants